Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve some issues with babel-plugin-ember-template-compilation v2.2.2+ #492

Merged

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Apr 24, 2024

The v2.2.2 release breaks our ember-template-imports v3 based setup.

We need to update to v4 to fix those and also fix the scope issues we accidentally depended on.

Draft since it seems to break the storybook setup. We probably need to update that first? Not sure what changed or what the issue was, but storybook seems to work now.

Fixes #491

TODO:

  • clean up the PR, lots of changes got in to test things but those aren't needed anymore.

@Windvis Windvis force-pushed the fix/babel-plugin-ember-template-compilation-v2.2.2-support branch from e4669d5 to b59a62a Compare May 8, 2024 09:13
@Windvis Windvis marked this pull request as ready for review May 8, 2024 09:17
@Windvis
Copy link
Contributor Author

Windvis commented May 8, 2024

I tested this out in an v4.12 app with ember-cli-babel v7 and ember-template-imports throws an exception because it expects ember-cli-babel v8. If that isn't a bug (we're not using template-imports in the app itself) I don't think we can release this as a bugfix.. And we weren't planning on releasing v4 yet either..

Edit: I think it's a bug in ember-template-imports which should be fixed by ember-cli/ember-template-imports#236

Discord discussion: https://discord.com/channels/480462759797063690/1239836417950220359

@Windvis Windvis force-pushed the fix/babel-plugin-ember-template-compilation-v2.2.2-support branch 2 times, most recently from 01cbeec to f243124 Compare May 13, 2024 13:04
index.js Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@Windvis Windvis changed the title Resolve some issues with babel-plugin-ember-template-compilation v2.2.2 Resolve some issues with babel-plugin-ember-template-compilation v2.2.2+ May 14, 2024
Windvis added 3 commits May 14, 2024 14:32
…2.2`

The v2.2.2+ releases breaks our ember-template-imports v3 based setup.

We need to update to v4 to fix those and also fix the scope issues we
accidentally depended on.

ember-template-imports v4 also requires some extra babel configuration
to ensure component imports aren't stripped out. This is the same setup
that the v2 blueprint uses as well.
@Windvis Windvis force-pushed the fix/babel-plugin-ember-template-compilation-v2.2.2-support branch from 92ce0e0 to 8360be7 Compare May 14, 2024 12:33
@Windvis
Copy link
Contributor Author

Windvis commented May 14, 2024

(Canary will fail until the fix is merged in ember-source, but we'll just ignore that since it will resolve itself eventually)

@Windvis Windvis merged commit c9a5a4d into master May 14, 2024
7 of 8 checks passed
@Windvis Windvis deleted the fix/babel-plugin-ember-template-compilation-v2.2.2-support branch May 14, 2024 12:48
@Windvis Windvis added bug Used when the PR fixes a bug included in a previous release. labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used when the PR fixes a bug included in a previous release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

babel-plugin-ember-template-compilation v2.2.2 incompatibilities
1 participant